Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[swss] add static anycast gateway support #3167

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

[swss] add static anycast gateway support #3167

wants to merge 18 commits into from

Conversation

yanjundeng
Copy link

@yanjundeng yanjundeng commented May 24, 2024 •
edited
Loading

What I did
Resync the original SAG support to latest master branch:
#1974

  • Add SAG support in intfmgr and intforch
  • Add support for SAG mac-address related IPv6 link-local toMe route
  • Add related testcases

Some improvements based on the original PR:

  • only add SAG-related IPv6 linklocal route for the VRF which has SAG-enabled interface
  • add SAG related FDB entry into linux bridge for the SAG-enabled vlan interface
  • Recover the vlan interface's mac in linux kernel to system mac once this SAG is disabled internally, for example unbind VRF or delete VRF case
  • fix some pytest issues in pipeline

Why I did it

  • To support static anycast gateway feature

How I verified it

Details if related

@yanjundeng
add static anycast gateway support
56c7dc6 
Signed-off-by: yanjun deng <yadeng@cisco.com>
@yanjundeng yanjundeng requested a review from prsunny as a code owner May 24, 2024 19:40
@yanjundeng
adjust intfsorch in test
a338736 
Signed-off-by: yanjun deng <yadeng@cisco.com>
@yanjundeng yanjundeng mentioned this pull request Jun 1, 2024
Merged
@yanjundeng
Copy link
Author

/azpw run Azure.sonic-swss

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-swss

@azure-pipelines Azure Pipelines
Copy link

Pull request contains merge conflicts.

@yanjundeng yanjundeng mentioned this pull request Jun 20, 2024
Open
prvattem
prvattem reviewed Sep 9, 2024
View reviewed changes
orchagent/p4orch/tests/fake_routeorch.cpp
@@ -0,0 +1,232 @@
extern "C"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add some description about the purpose of fake_routeorch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@prvattem prvattem prvattem left review comments

@prsunny prsunny Awaiting requested review from prsunny prsunny is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yanjundeng @mssonicbld @prvattem